Skip to content

Conversation

@bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Dec 5, 2025

Instead of only building the tests, execute them as well

Fixes: #9427
Issue: rdar://165491718

Depends on: #9406

@bkhouri bkhouri added the continuous integration/deployment Changes to the Continuous Integration/Deployment system/pipelines label Dec 5, 2025
@bkhouri bkhouri force-pushed the t/main/gh9427_rdar165491718_enable_testing_in_gha branch 7 times, most recently from 81fce8b to bc9daf1 Compare December 12, 2025 02:05
@bkhouri bkhouri force-pushed the t/main/gh9427_rdar165491718_enable_testing_in_gha branch 2 times, most recently from c049c75 to 7c97af9 Compare December 19, 2025 18:30
Instead of only building the tests, execute them as well.  Also run against
swiftbuild backend.

Fixes: swiftlang#9427
Issue: rdar://165491718
@bkhouri bkhouri force-pushed the t/main/gh9427_rdar165491718_enable_testing_in_gha branch from 7c97af9 to 0472150 Compare December 19, 2025 19:32
@bkhouri
Copy link
Contributor Author

bkhouri commented Dec 19, 2025

Since the PR modifies Utilities/build-using-self, which is used by the test hosted pipelines. let's trigger them.

@swift-ci test self hosted

"swift-test",
*global_args,
"--vv",
# "--vv",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: why turn off very verbose here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find it significantly increases the log, making it difficult for view the console logs in Jenkins - not sure how GitHub Actions handles the larges logs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, it's my understanding it was added by @daveinglis to help troubleshoot CI build failure in a PR.

@bkhouri bkhouri force-pushed the t/main/gh9427_rdar165491718_enable_testing_in_gha branch 2 times, most recently from 9a0fdf5 to 57222a9 Compare December 20, 2025 01:34
@bkhouri bkhouri force-pushed the t/main/gh9427_rdar165491718_enable_testing_in_gha branch from 57222a9 to 33b3f73 Compare December 20, 2025 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous integration/deployment Changes to the Continuous Integration/Deployment system/pipelines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Enable testing in GitHub Actions

3 participants